Skip to content

* Fixed an error in converting foreign key values to integers#173

Open
dohnal wants to merge 1 commit intonette:masterfrom
dohnal:master
Open

* Fixed an error in converting foreign key values to integers#173
dohnal wants to merge 1 commit intonette:masterfrom
dohnal:master

Conversation

@dohnal
Copy link

@dohnal dohnal commented Jun 17, 2017

  • bug fix? yes
  • new feature? no
  • BC break? yes/no
  • doc PR: nette/docs#???

An issue of auto-convert of row values used as foreign keys to integers (by PHP). The values were used in cacheKeys as array keys, so they got converted by PHP. It is clearly wrong. This request changes the storage of row values to array values.

@dg
Copy link
Member

dg commented Jun 19, 2017

It seems that it breaks tests.

ping @dohnal

@dg dg force-pushed the master branch 4 times, most recently from abe9849 to 9358712 Compare June 29, 2017 15:38
@dg dg force-pushed the master branch 7 times, most recently from f1a9aa2 to ec7a207 Compare July 26, 2017 21:39
@dg dg force-pushed the master branch 14 times, most recently from 1ea2fe2 to f4d7fc0 Compare March 23, 2018 16:52
@dg dg force-pushed the master branch 4 times, most recently from 9b70c0e to dfb79e3 Compare March 17, 2019 06:22
@dg dg force-pushed the master branch 3 times, most recently from cba566a to 302586e Compare April 1, 2019 04:56
@dg dg force-pushed the master branch 2 times, most recently from 4c0e8d0 to e17242a Compare November 19, 2019 18:28
@dg dg force-pushed the master branch 6 times, most recently from f9fc19d to dd75e74 Compare December 13, 2019 18:06
@dg dg force-pushed the master branch 2 times, most recently from cce6a9c to 7e705c5 Compare December 27, 2019 03:53
@dg dg force-pushed the master branch 2 times, most recently from 9acf825 to a268c60 Compare March 27, 2020 11:52
@dg dg force-pushed the master branch 2 times, most recently from a56e7fe to d9b5f29 Compare June 9, 2020 14:19
@dg dg force-pushed the master branch 2 times, most recently from f016c33 to 6596c77 Compare July 10, 2020 16:30
@dg dg force-pushed the master branch 2 times, most recently from a9e5763 to c3cc31a Compare October 6, 2020 17:24
@dg dg force-pushed the master branch 2 times, most recently from e91b234 to 953502b Compare October 15, 2020 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants